-
Notifications
You must be signed in to change notification settings - Fork 87
Add v1.3 release notes. #344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (2)
release_notes/v1.3.md:2
- The TODO placeholder for the number of pull requests may confuse users. Consider updating this placeholder with the final count before release.
We are excited to announce the release of OPEA version 1.3, which includes significant contributions from the open-source community. This release addresses over 400(TODO: the number is subject to change) pull requests.
release_notes/RELEASE.md:18
- [nitpick] The date format seems inconsistent with other entries (e.g., using 'Apr' instead of 'July'). Consider standardizing abbreviations to maintain consistency in the release schedule.
| 1.4 | July 2025 |
mkbhanda
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does AMD want to add to the table of supported models on serving frameworks and hardware?
Arc not mentioned in table, do we want to add?
Yu-amd
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AMD team has a validated model matrix we can provide.
@joshuayao Please see the table above, which includes a column of validated models on ROCm. Thanks! |
ashahba
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Hi @Yu-amd Updated. Thanks for your information. |
Co-authored-by: Eero Tamminen <eero.t.tamminen@intel.com>
Yu-amd
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you.
eero-t
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I had typos in my suggestions.
Co-authored-by: Eero Tamminen <eero.t.tamminen@intel.com>
No problem. Thanks for the updates. |
eero-t
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess "Other Notable Changes" section content came directly from the PR titles, but for the release notes I think it would good to use correct & consistent capitalization for the names (and maybe also monotype for function names).
Following terms seemed to have instances with incorrect capitalization:
- AgentQnA
- Dataprep
- DeepSeek
- Docker
- Helm
- LlamaGuard
- LVM
- Milvus
- MySQL
- OpenAI
- PubMed
- Python
- RAG
- TEI
- vLLM
- Xeon
- YAML
Co-authored-by: Eero Tamminen <eero.t.tamminen@intel.com>


Add v1.3 release notes.